Skip to content

fix: remove stale slicing type ignore - #462

Open
mkg0908 wants to merge 1 commit into
alchemistry:masterfrom
mkg0908:fix-460-remove-unused-ignore
Open

fix: remove stale slicing type ignore#462
mkg0908 wants to merge 1 commit into
alchemistry:masterfrom
mkg0908:fix-460-remove-unused-ignore

Conversation

@mkg0908

@mkg0908 mkg0908 commented Jun 2, 2026

Copy link
Copy Markdown

Summary

  • remove the unused type: ignore[misc] suppression from slicing()
  • allow the mypy unused-ignore check to pass for the reported line

Tests

  • .venv/bin/mypy src/alchemlyb/preprocessing/subsampling.py --ignore-missing-imports --disable-error-code no-any-return
  • .venv/bin/python -m compileall -q src/alchemlyb/preprocessing/subsampling.py
  • git diff --check

Closes #460

@codecov

codecov Bot commented Jun 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.01%. Comparing base (5c029f5) to head (d60b903).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #462   +/-   ##
=======================================
  Coverage   99.01%   99.01%           
=======================================
  Files          29       29           
  Lines        2446     2446           
  Branches      464      464           
=======================================
  Hits         2422     2422           
  Misses          2        2           
  Partials       22       22           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@xiki-tempula

Copy link
Copy Markdown
Collaborator

Would it be better if you could fix the other mypy and ruff issue, please? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix mypy type check

2 participants